-
-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SF-3077 Add UI for mixed training sources #2875
base: feature/SF-2900
Are you sure you want to change the base?
Conversation
d49a457
to
4bd2738
Compare
Observations:
|
1ac6820
to
d58cf38
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feature/SF-2900 #2875 +/- ##
===================================================
+ Coverage 79.86% 79.90% +0.04%
===================================================
Files 534 534
Lines 31014 31082 +68
Branches 5014 5050 +36
===================================================
+ Hits 24768 24836 +68
Misses 5496 5496
Partials 750 750 ☔ View full report in Codecov by Sentry. |
4bd2738
to
ea8054e
Compare
d58cf38
to
b9268e3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll also want to add an optional title parameter to the book selection component, which contains the bulk select checkboxes. Then place the project name on the same line as the checkboxes.
Reviewable status: 0 of 14 files reviewed, all discussions resolved
@RaymondLuong3 I should note that when selecting books in bulk, it seems that the first time yous select them it selects correctly, but then upon unselecting and re-selecting it doesn't. |
b9268e3
to
231c790
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for finding that. I figured out the main issue was that arrays were being passed around by reference rather than by values. I've cleaned it up and now it works. I've also addressed the extra training books notice and the project label for the multi-book-select component.
Reviewable status: 0 of 16 files reviewed, all discussions resolved
b6d0947
to
deef1aa
Compare
ea8054e
to
cc508cd
Compare
deef1aa
to
8a007bb
Compare
e6895e4
to
6462ced
Compare
8a007bb
to
f2f1d2d
Compare
This PR adds the ability to mix in training data from multiple sources. See the balsamiq for the original design.
No training books selected
Training books selected
This change is